Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

soundwire: intel_auxdevice: add kernel parameter for mclk divider #5067

Merged
merged 1 commit into from
Jun 19, 2024
Merged

soundwire: intel_auxdevice: add kernel parameter for mclk divider #5067

merged 1 commit into from
Jun 19, 2024

Conversation

plbossart
Copy link
Member

Add a kernel parameter to work-around discrepancies between hardware and platform firmware, it's not unusual to see e.g. 38.4MHz listed in _DSD properties as the SoundWire clock source, but the hardware may be based on a 19.2 MHz mclk source.

@plbossart
soundwire: intel_auxdevice: add kernel parameter for mclk divider
416a639 
Add a kernel parameter to work-around discrepancies between hardware
and platform firmware, it's not unusual to see e.g. 38.4MHz listed in
_DSD properties as the SoundWire clock source, but the hardware may be
based on a 19.2 MHz mclk source.

Signed-off-by: Pierre-Louis Bossart <[email protected]>
@plbossart
Copy link
Member Author

This is weird

https://sof-ci.01.org/linuxpr/PR5067/build3697/devicetest/index.html?model=LNLM_SDW_AIOC&testcase=multiple-pause-resume-50

2024-06-17 13:31:15 UTC [REMOTE_INFO] pipeline: Amp feedback with arecord
2024-06-17 13:31:15 UTC [REMOTE_INFO] pipeline: Jack In with arecord
2024-06-17 13:31:15 UTC [REMOTE_INFO] Check expect exit status
2024-06-17 13:31:15 UTC [REMOTE_INFO] Amp feedback to command: arecord -D hw:0,3 -r 48000 -c 4 -f S16_LE -vv -i /dev/null -q
spawn arecord -D hw:0,3 -r 48000 -c 4 -f S16_LE -vv -i /dev/null -q
2024-06-17 13:31:15 UTC [REMOTE_INFO] Deepbuffer Jack Out to command: aplay -D hw:0,31 -r 48000 -c 2 -f S16_LE -vv -i /dev/zero -q
arecord: main:831: audio open error: Device or resource busy
2024-06-17 13:31:15 UTC [REMOTE_INFO] wait for expect process finished
spawn aplay -D hw:0,31 -r 48000 -c 2 -f S16_LE -vv -i /dev/zero -q
Hardware PCM card 0 'sof-soundwire' device 31 subdevice 0

First time I see this one @bardliao @fredoh9 @marc-hb

@plbossart
Copy link
Member Author

SOFCI TEST

@bardliao
Copy link
Collaborator

This is weird

https://sof-ci.01.org/linuxpr/PR5067/build3697/devicetest/index.html?model=LNLM_SDW_AIOC&testcase=multiple-pause-resume-50

2024-06-17 13:31:15 UTC [REMOTE_INFO] pipeline: Amp feedback with arecord
2024-06-17 13:31:15 UTC [REMOTE_INFO] pipeline: Jack In with arecord
2024-06-17 13:31:15 UTC [REMOTE_INFO] Check expect exit status
2024-06-17 13:31:15 UTC [REMOTE_INFO] Amp feedback to command: arecord -D hw:0,3 -r 48000 -c 4 -f S16_LE -vv -i /dev/null -q
spawn arecord -D hw:0,3 -r 48000 -c 4 -f S16_LE -vv -i /dev/null -q
2024-06-17 13:31:15 UTC [REMOTE_INFO] Deepbuffer Jack Out to command: aplay -D hw:0,31 -r 48000 -c 2 -f S16_LE -vv -i /dev/zero -q
arecord: main:831: audio open error: Device or resource busy
2024-06-17 13:31:15 UTC [REMOTE_INFO] wait for expect process finished
spawn aplay -D hw:0,31 -r 48000 -c 2 -f S16_LE -vv -i /dev/zero -q
Hardware PCM card 0 'sof-soundwire' device 31 subdevice 0

First time I see this one @bardliao @fredoh9 @marc-hb

But this is not possible caused by this PR, right? We didn't set the parameter in CI devices.

@plbossart
Copy link
Member Author

and now alsa-bat issues haha.

https://sof-ci.01.org/linuxpr/PR5067/build3722/devicetest/index.html?model=LNLM_SDW_AIOC&testcase=check-alsabat-headset-capture-599

@plbossart plbossart merged commit 414bda9 into thesofproject:topic/sof-dev Jun 19, 2024
12 of 15 checks passed
@marc-hb marc-hb mentioned this pull request Jun 20, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bardliao bardliao bardliao approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@plbossart @bardliao